-
-
Notifications
You must be signed in to change notification settings - Fork 39
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
series: add exercise #346
series: add exercise #346
Conversation
The R track has a particularly fierce linter. We all get caught by this the first few times! This looks like a good addition, once we placate the linter. |
981cb51
to
988330e
Compare
I've updated the PR |
"uuid": "2ef6f027-a27c-4b00-bf1e-2969baf04a36", | ||
"practices": [], | ||
"prerequisites": [], | ||
"difficulty": 3 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"difficulty": 3 | |
"difficulty": 3, | |
"topics": [ | |
"strings" | |
] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm fine adding this, but you do know that topics
aren't used by the website, right?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Haha, it did cross my mind that if you hadn't included it, it was quite likely because it's not really needed or being used anywhere, but was more efficient for me to suggest it and wait and see how you responded than for me to try and check if or how it might be used later. If there's no value in adding it (now or for later) then obviously fine with leaving it out.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
No description provided.